Skip to content

Conversation

@enghitalo
Copy link
Contributor

@enghitalo enghitalo commented Dec 5, 2025

wrk --connection 512 --threads 16 --duration 60s http://localhost:3000
Running 1m test @ http://localhost:3000
  16 threads and 512 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.21ms    1.43ms  20.84ms   84.97%
    Req/Sec    29.61k     4.04k   53.96k    78.26%
  28302709 requests in 1.00m, 2.06GB read
Requests/sec: 471002.42
Transfer/sec:     35.04MB

@enghitalo
Copy link
Contributor Author

enghitalo commented Dec 5, 2025

@spytheman sorry. Can you help me solve this CI error? It's compiling files one by one instead of the whole folder

@enghitalo
Copy link
Contributor Author

In the next PR I'll make the new_servers use the structure in the parameter instead of several different parameters. Since this library wasn't working, I don't think this will be a breaking change.

@spytheman
Copy link
Member

@spytheman sorry. Can you help me solve this CI error? It's compiling files one by one instead of the whole folder

Edit the start of cmd/tools/vbuild-examples.v and add examples/fasthttp/ there too.

@spytheman
Copy link
Member

I give up reviewing this - it is too big, and also breaking an existing API :-| .

@medvednikov merge this if you want, but imho it may be a good idea to reduce the amount of fine http server implementations and web frameworks in vlib, to something manageable and maintainable, instead of having several, following different styles, made by different people, and at different levels of completion.

@enghitalo
Copy link
Contributor Author

I didn't consider the changes in new_server as a breaking change because the fasthttp library wasn't working until now.

@enghitalo
Copy link
Contributor Author

enghitalo commented Dec 10, 2025

I'm going to turn this into a draft now with the intention of getting new_veb working and creating tests to prevent rollbacks in this case. I will do it super fast

@spytheman spytheman marked this pull request as draft December 10, 2025 16:33
@enghitalo enghitalo marked this pull request as ready for review December 10, 2025 19:05
@enghitalo
Copy link
Contributor Author

enghitalo commented Dec 10, 2025

New Veb

./v -d new_veb -prod crun examples/veb/veb_example.v 
wrk -t16 -c512 -d30s http://127.0.0.1:8082/show_text
Running 30s test @ http://127.0.0.1:8082/show_text
  16 threads and 512 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    20.17ms   34.45ms 280.10ms   83.45%
    Req/Sec     5.38k     2.13k   36.55k    69.51%
  2568889 requests in 30.09s, 240.09MB read
Requests/sec:  85368.38
Transfer/sec:      7.98MB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants